Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[azure-pipelines | analyzer] Code coverage integration #110

Merged
merged 15 commits into from
May 12, 2020
Merged

[azure-pipelines | analyzer] Code coverage integration #110

merged 15 commits into from
May 12, 2020

Conversation

Raffaello
Copy link
Owner

@Raffaello Raffaello commented May 10, 2020 •
edited
Loading

  • remove mingw pipeline

  • try with a linux container on a windows machine

  • Clang code coverage

  • misssing llvm-cov on azure osx image... ???

@Raffaello Raffaello added documentation Improvements or additions to documentation configuration cmake or project related code, configuaration refactor improve the code without enhancing in functionalities labels May 10, 2020
@Raffaello Raffaello added this to the Better CI milestone May 10, 2020
@Raffaello Raffaello self-assigned this May 10, 2020
@Raffaello Raffaello changed the title split variable file [azure-pipelines | analyzer] Code coverage integartion May 10, 2020
@Raffaello Raffaello linked an issue May 10, 2020 that may be closed by this pull request
[azure-pipelines | analyzer] Integrate Code Coverage #106
Closed
8 tasks
@Raffaello Raffaello force-pushed the 106-code-cov branch 8 times, most recently from ed2e181 to 1a94e6d Compare May 10, 2020 18:50
@Raffaello Raffaello changed the title [azure-pipelines | analyzer] Code coverage integartion [azure-pipelines | analyzer] Code coverage integration May 10, 2020
@Raffaello Raffaello force-pushed the 106-code-cov branch 3 times, most recently from de3aef6 to 84e5ae9 Compare May 10, 2020 22:23
@Raffaello Raffaello force-pushed the 106-code-cov branch 6 times, most recently from 3e2f271 to d2a0a1f Compare May 12, 2020 16:50
Repository owner deleted a comment from codecov bot May 12, 2020
@Raffaello Raffaello force-pushed the 106-code-cov branch 2 times, most recently from f014958 to d3bcde6 Compare May 12, 2020 17:11
@Raffaello
Copy link
Owner Author

sonar cloud still misterious:

WARN: File not analysed by Sonar, so ignoring coverage: ...

INFO: Sensor gcov [cpp] (done) | time=319ms

@Raffaello Raffaello force-pushed the 106-code-cov branch 2 times, most recently from be25ab0 to a43d410 Compare May 12, 2020 17:41
@Raffaello
Copy link
Owner Author

sonar cloud wont upload gcov reports.... ignoring for this time

@Raffaello Raffaello force-pushed the 106-code-cov branch 3 times, most recently from 47917f4 to 2555207 Compare May 12, 2020 17:57
@Raffaello Raffaello marked this pull request as ready for review May 12, 2020 18:11
@codecov
Copy link

codecov bot commented May 12, 2020 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (master@79fdfc6). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #110   +/-   ##
=========================================
  Coverage          ?   81.63%           
=========================================
  Files             ?        2           
  Lines             ?      294           
  Branches          ?        0           
=========================================
  Hits              ?      240           
  Misses            ?       54           
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 79fdfc6...ecb0d75. Read the comment docs.

Raffaello
Raffaello commented May 12, 2020
View reviewed changes
appveyor.yml Outdated Show resolved Hide resolved
Raffaello
Raffaello commented May 12, 2020
View reviewed changes
appveyor.yml Outdated Show resolved Hide resolved
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@Raffaello Raffaello merged commit c78af53 into master May 12, 2020
@Raffaello Raffaello deleted the 106-code-cov branch May 12, 2020 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Raffaello Raffaello

Labels
configuration cmake or project related code, configuaration documentation Improvements or additions to documentation refactor improve the code without enhancing in functionalities
Projects
None yet
Milestone
Better CI
Development

Successfully merging this pull request may close these issues.

[azure-pipelines | analyzer] Integrate Code Coverage Clang compile options
1 participant
@Raffaello